Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

teams: smoother survey submissions (fixes #5348) #5349

Merged
merged 6 commits into from
Feb 26, 2025

Conversation

Okuro3499
Copy link
Collaborator

fixes #5348
Focus on getting the gender and age post survey, hide other fields but can still be filled
Screen_recording_20250225_174940.webm

@Okuro3499 Okuro3499 marked this pull request as ready for review February 25, 2025 15:26
@huitk
Copy link
Member

huitk commented Feb 25, 2025

Is it possible that hide and show additional button be on the left of update?

@Okuro3499
Copy link
Collaborator Author

Is it possible that hide and show additional button be on the left of update?

@huitk
Screen_recording_20250226_134930.webm

pavi38
pavi38 previously requested changes Feb 26, 2025
Copy link
Member

@pavi38 pavi38 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Okuro3499 I can enter alphabets in phone number field and the calendar don't pop up for the birth date. And level and language dropdowns are not dark mode compatible.

Screen.Recording.2025-02-26.at.11.46.56.AM.mov

@dogi
Copy link
Member

dogi commented Feb 26, 2025

@pavi38 thanks -> create new issue

@dogi dogi dismissed pavi38’s stale review February 26, 2025 20:36

good catch -> new PR

@dogi dogi changed the title teams: smoother survey submission (fixes #5348) teams: smoother survey submissions (fixes #5348) Feb 26, 2025
@dogi dogi merged commit ea52c67 into master Feb 26, 2025
2 checks passed
@Okuro3499 Okuro3499 deleted the 5348-surveys-custom-post-survey-form branch February 28, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

surveys: custom post survey form
6 participants